-
Notifications
You must be signed in to change notification settings - Fork 985
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: handle deep link too early when onboarding #18363
Conversation
Jenkins BuildsClick to see older builds (47)
|
e7f685f
to
c133778
Compare
77% of end-end tests have passed
Failed tests (6)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
Expected to fail tests (5)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Passed tests (37)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
c133778
to
9b842c8
Compare
c8dd6be
to
0e2c3ef
Compare
0e2c3ef
to
9f5f034
Compare
✔️ status-mobile/prs/android/PR-18363#8 🔹 ~6 min 54 sec 🔹 9f5f034 🔹 📦 android package |
0% of end-end tests have passed
Failed tests (3)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
9f5f034
to
f253c02
Compare
67% of end-end tests have passed
Failed tests (1)Click to expandClass TestActivityMultipleDevicePR:
Expected to fail tests (1)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Passed tests (4)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
f253c02
to
0d140be
Compare
100% of end-end tests have passed
Passed tests (1)Click to expandClass TestActivityMultipleDevicePR:
|
Hi @yqrashawn! Thanks for the PR! ISSUE 1 : [Android] User is not redirected to the Status app after tap on community universal link Steps:
Actual result: user see a community page opened in a mobile browser link_issue.mp4It repoduces on PR builds. Nightly doesn't have this issue. Expected result: user should be able to be redirected to a Status App actual_result.mp4 |
Hi @mariia-skrypnyk, can you try paste the link into chrome browser see if it opens up status app? |
Thanks for you comment @yqrashawn! I double-checked it and it has the same result as from the Telegram. |
Hi @yqrashawn! I see you've maid a new builds! |
0d140be
to
83884da
Compare
Hi @mariia-skrypnyk |
Tested in this PR:
Great job @yqrashawn, PR can be merged! |
0e7b5c3
to
5adaa52
Compare
5adaa52
to
14d2da6
Compare
fixes #18074
Summary
there's a
(get db :onboarding-2/new-account?)
instatus-mobile/src/status_im/contexts/profile/login/events.cljs
Line 144 in e7f685f
updated to
(get db :onboarding/new-account?)
so that handle deep link won't be triggerd until user press the button an the end of the onboarding flow
status: ready